Skip to content

ci(min-deps): promote min-deps to first-class CI alongside latest-deps#1480

Merged
sd-db merged 3 commits into
mainfrom
sd-db/feat/min-deps-first-class-ci
May 26, 2026
Merged

ci(min-deps): promote min-deps to first-class CI alongside latest-deps#1480
sd-db merged 3 commits into
mainfrom
sd-db/feat/min-deps-first-class-ci

Conversation

@sd-db
Copy link
Copy Markdown
Collaborator

@sd-db sd-db commented May 25, 2026

Summary

Promotes min-deps to a first-class CI citizen alongside latest-deps so lower-bound regressions surface pre-merge on every PR — including forks. Shared dep cache now covers both resolutions; min-deps unit + build run on every PR; min-deps integration runs nightly + on /integration-test min-deps.

Test plan

  • Warm-cache populates new combined-key shape and lowest-direct wheel closure.
  • Cache restore wired correctly (branch-scope dispatch run hits cache, JFrog skipped).
  • PR checks green on main.yml + main-min-deps.yml.
  • /integration-test still routes to integration.yml.

Post-merge: dispatch warmDepsCache.yml on main once to populate main-scope cache immediately. Need to check /integration-test min-deps routing

- setup-python-deps + warmDepsCache: cache key hash includes
  requirements.lowest-direct.txt; warm job creates min-deps + verify-min-deps
  envs and downloads the lowest-direct closure into the wheelhouse so fork
  PRs can run min-deps tests fully offline from cache.
- main-min-deps.yml (renamed from min-deps-test-fast.yml): mirrors main.yml
  shape — cache-restore + JFrog fallback, concurrency with cancel-in-progress,
  paths-ignore for *.md. Fork-skip guards removed; cache makes forks work.
- integration-min-deps.yml (renamed from min-deps-test-slow.yml): adds
  workflow_dispatch with pr_numbers / git_ref inputs, full target parser, and
  a report-status job that posts a result comment back to the PR — mirroring
  integration.yml for the slash-command path.
- integration-trigger.yml: parses the first arg after /integration-test.
  /integration-test min-deps routes to integration-min-deps.yml; anything
  else falls through to integration.yml (free-form trailing context).
@sd-db sd-db requested a review from jprakash-db as a code owner May 25, 2026 07:22
@sd-db
Copy link
Copy Markdown
Collaborator Author

sd-db commented May 25, 2026

/integration-test min-deps

@sd-db
Copy link
Copy Markdown
Collaborator Author

sd-db commented May 25, 2026

/integration-test

@github-actions
Copy link
Copy Markdown

Integration tests dispatched for PR #1480 by @sd-db. Track progress in the Actions tab.

1 similar comment
@github-actions
Copy link
Copy Markdown

Integration tests dispatched for PR #1480 by @sd-db. Track progress in the Actions tab.

@github-actions
Copy link
Copy Markdown

Integration results for PR #1480 — UC cluster ❌ cancelled · SQL warehouse ❌ cancelled · All-purpose cluster ❌ cancelled · Shard coverage ❌ cancelled

Run details.

@github-actions
Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

@github-actions
Copy link
Copy Markdown

Integration results for PR #1480 — UC cluster ❌ failure · SQL warehouse ❌ failure · All-purpose cluster ✅ success · Shard coverage ✅ success

Run details.

@sd-db
Copy link
Copy Markdown
Collaborator Author

sd-db commented May 26, 2026

/integration-test

@github-actions
Copy link
Copy Markdown

Integration tests dispatched for PR #1480 by @sd-db. Track progress in the Actions tab.

@github-actions
Copy link
Copy Markdown

Integration results for PR #1480 — UC cluster ✅ success · SQL warehouse ✅ success · All-purpose cluster ✅ success · Shard coverage ✅ success

Run details.

@sd-db sd-db merged commit b90fb0a into main May 26, 2026
12 checks passed
@sd-db sd-db deleted the sd-db/feat/min-deps-first-class-ci branch May 26, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants